Refresh check_buck_targets.sh #13196
Conversation
|
@mszeszko-meta has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@mszeszko-meta has updated the pull request. You must reimport the pull request before landing. |
mszeszko-meta
force-pushed
the
refresh_check_buck_targets_script
branch
from
3bcee43 to
f34cbd8
Compare
|
@mszeszko-meta has updated the pull request. You must reimport the pull request before landing. |
|
@mszeszko-meta has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@mszeszko-meta has updated the pull request. You must reimport the pull request before landing. |
|
@mszeszko-meta has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@mszeszko-meta has updated the pull request. You must reimport the pull request before landing. |
|
@mszeszko-meta has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
mszeszko-meta
force-pushed
the
refresh_check_buck_targets_script
branch
from
3138650 to
b4af0b9
Compare
|
@mszeszko-meta has updated the pull request. You must reimport the pull request before landing. |
|
@mszeszko-meta has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@mszeszko-meta merged this pull request in bfe6cc2. |
Summary:
This is a followup to #13190. We're patching the targets generating script to construct
BUCKfile instead of deprecatedTARGETSfile + adding safety checks to ensure thatBUCKfile does not go missing (either as a direct renaming / removal OR as a modification to buckfier's script(s)).Test plan:
-> Simulate removing
BUCKfile via commit-> Simulate buckifier script removing the
BUCKfile